Skip to content

in_calyptia_fleet: backport support for net.* properties for the upstream connection#11025

Closed
mattyaterz wants to merge 1 commit intofluent:4.0from
mattyaterz:matty-fix-custom-calyptia-input-test-init
Closed

in_calyptia_fleet: backport support for net.* properties for the upstream connection#11025
mattyaterz wants to merge 1 commit intofluent:4.0from
mattyaterz:matty-fix-custom-calyptia-input-test-init

Conversation

@mattyaterz
Copy link

@mattyaterz mattyaterz commented Oct 15, 2025

Copying #10998 (which went into master) into 4.0

Enter [N/A] in the box, if an item is not applicable to your change.

Testing
Before we can approve your change; please submit the following in a comment:

  • [N/A] Example configuration file for the change
  • Debug log output from testing the change

Before

matty@ubuntu:~/dev/fluent-bit/build$ ./bin/flb-rt-custom_calyptia_input_test 
Test set_fleet_input_properties...                Test interrupted by SIGSEGV.
Test machine_id_generation...                     Test interrupted by SIGSEGV.
FAILED: 2 of 2 unit tests have failed.

After

matty@ubuntu:~/dev/fluent-bit/build$ ./bin/flb-rt-custom_calyptia_input_test 
Test set_fleet_input_properties...              [ OK ]
Test machine_id_generation...                   [ OK ]
SUCCESS: All unit tests have passed.
  • [N/A] Attached Valgrind output that shows no leaks or memory corruption was found

If this is a change to packaging of containers or native binaries then please confirm it works for all targets.

  • [N/A] Run local packaging test showing all targets (including any new ones) build.
  • [N/A] Set ok-package-test label to test for all targets (requires maintainer to do).

Documentation

  • [N/A] Documentation required for this feature

Backporting

  • [N/A] Backport to latest stable release.

Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.

@coderabbitai
Copy link

coderabbitai bot commented Oct 15, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mattyaterz mattyaterz force-pushed the matty-fix-custom-calyptia-input-test-init branch from 4771c63 to 6c17307 Compare October 15, 2025 18:21
@mattyaterz mattyaterz changed the title tests/runtime/custom_calyptia_input: initialize flb_custom_instance n… tests: runtime: custom_calyptia_input: initialize flb_custom_instance n… Oct 15, 2025
@mattyaterz mattyaterz changed the title tests: runtime: custom_calyptia_input: initialize flb_custom_instance n… tests: runtime: custom_calyptia_input: initialize flb_custom_instance net_properties in init_test_context() Oct 15, 2025
@mattyaterz mattyaterz force-pushed the matty-fix-custom-calyptia-input-test-init branch from 6c17307 to 89c46e0 Compare October 15, 2025 18:48
@mattyaterz mattyaterz changed the title tests: runtime: custom_calyptia_input: initialize flb_custom_instance net_properties in init_test_context() in_calyptia_fleet: backport support for net.* properties for the upstream connection Oct 15, 2025
@mattyaterz
Copy link
Author

mattyaterz commented Oct 15, 2025

Closing as a duplicate of #11001

@mattyaterz mattyaterz closed this Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@edsiper edsiper Awaiting requested review from edsiper edsiper is a code owner

@fujimotos fujimotos Awaiting requested review from fujimotos

@koleini koleini Awaiting requested review from koleini koleini is a code owner

@pwhelan pwhelan Awaiting requested review from pwhelan pwhelan is a code owner

@patrick-stephens patrick-stephens Awaiting requested review from patrick-stephens patrick-stephens is a code owner

@niedbalski niedbalski Awaiting requested review from niedbalski niedbalski is a code owner

Assignees

No one assigned

Labels

docs-required

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mattyaterz